Skip to content

Fix multi-join dynamic filtering#15057

Merged
highker merged 1 commit intoprestodb:masterfrom
kewang1024:Fix_multi_join_dynamic_filtering
Aug 21, 2020
Merged

Fix multi-join dynamic filtering#15057
highker merged 1 commit intoprestodb:masterfrom
kewang1024:Fix_multi_join_dynamic_filtering

Conversation

@kewang1024
Copy link
Copy Markdown
Collaborator

== NO RELEASE NOTE ==

@kewang1024 kewang1024 force-pushed the Fix_multi_join_dynamic_filtering branch from 134e295 to d48a77d Compare August 20, 2020 09:11
@kewang1024 kewang1024 marked this pull request as ready for review August 20, 2020 09:18
@kewang1024 kewang1024 force-pushed the Fix_multi_join_dynamic_filtering branch from d48a77d to f164133 Compare August 20, 2020 09:26
Copy link
Copy Markdown

@highker highker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add more tests to this commit?

@kewang1024 kewang1024 force-pushed the Fix_multi_join_dynamic_filtering branch 2 times, most recently from e4cac04 to 9b1636e Compare August 20, 2020 17:24
@kewang1024 kewang1024 requested a review from highker August 20, 2020 19:11
Copy link
Copy Markdown

@highker highker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nits only

@highker highker self-assigned this Aug 20, 2020
cherry pick from trinodb/trino#2659

Co-Authored-By: Roman Zeyde <zeyde@varada.io>
@kewang1024 kewang1024 force-pushed the Fix_multi_join_dynamic_filtering branch from 9b1636e to 91c9b4b Compare August 21, 2020 00:17
@highker highker merged commit f25aa0c into prestodb:master Aug 21, 2020
@kewang1024 kewang1024 deleted the Fix_multi_join_dynamic_filtering branch August 22, 2020 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants